Conversation
|
@nyandika is attempting to deploy a commit to the React Developers Kenya Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Working on the failing tests. Will send an update |
|
@orama254 Could you authorize this PR to deploy to vercel to get the pre-release view |
|
@greptile |
Greptile SummaryThis PR updates dependencies to address Next.js vulnerabilities, bumping Next.js from 12.3.1 to 16.1.6, React from 18.1.0 to 19.2.4, and modernizing the build toolchain. The Key changes:
Critical issue:
Confidence Score: 2/5
|
| Filename | Overview |
|---|---|
| package.json | Major version bumps for Next.js (12.3.1 → 16.1.6), React (18.1.0 → 19.2.4), and build tools; lint script simplified |
| tailwind.config.js | Duplicate extend object at lines 6 and 16 - second will override first, losing initial animation/fontFamily config |
| eslint.config.mjs | New ESLint flat config for Next.js with proper global ignores |
| next.config.js | Updated image config to use remotePatterns (replacing deprecated domains), removed eslint.dirs |
| tsconfig.json | Changed JSX from preserve to react-jsx for React 19 compatibility |
| .github/workflows/validate.yml | Added prod branch trigger, updated Node.js 14.x → 24.x, updated actions to v6 |
| .github/workflows/e2e.yml | Added prod branch trigger, updated Node.js 18.x → 24.x, updated actions to v5/v6 |
| src/components/HeroHeader/HeroHeader.tsx | Added requestAnimationFrame for initial width setting; removed unused React import; cleanup added to useEffect |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Dependency Updates] --> B[Next.js 12.3.1 → 16.1.6]
A --> C[React 18.1.0 → 19.2.4]
A --> D[Tailwind 3.1.2 → 4.2.0]
B --> E[New ESLint Flat Config]
B --> F[Update next.config.js APIs]
C --> G[Change TSConfig jsx to react-jsx]
C --> H[Remove React imports]
D --> I[Update PostCSS plugin]
D --> J[Update tailwind.config.js]
E --> K[Run Prettier/Linting]
F --> K
G --> K
H --> K
I --> K
J --> K
K --> L[38 Files Changed]
J -.-> M[⚠️ Duplicate extend objects]
style M fill:#ff6b6b,stroke:#c92a2a,color:#fff
style A fill:#4dabf7,stroke:#1971c2,color:#fff
style L fill:#51cf66,stroke:#2f9e44,color:#fff
Last reviewed commit: 41570df
|
@orama254 fixed issues brought about by merge from prod |
|
@greptile |
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
useState<number | null>(() => typeof window !== 'undefined' ? window.innerWidth : null) — initializes the value immediately via a lazy initializer instead of inside the effect typeof window !== 'undefined' guard prevents SSR errors during Next.js server rendering
Changes proposed
There have been recent vulnerabilities in the nextjs space. Taking this chance to bump up all package versions
Also added prod branch to github actions. This necessisated a rerun of prettier and linting hence the other numerous file changes
Check List (Check all the applicable boxes)
Screenshots
N/A
Note to reviewers
N/A